Skip to content

Dev0#2

Open
iamserda wants to merge 6 commits intomainfrom
dev0
Open

Dev0#2
iamserda wants to merge 6 commits intomainfrom
dev0

Conversation

@iamserda
Copy link
Copy Markdown
Owner

This pull request introduces several major improvements to the project, including the addition of a calculator module with tests, a robust file information utility with tests, enhancements to the development workflow (Makefile, pre-commit, and dependencies), and improvements to documentation and environment configuration. These changes collectively improve the project's functionality, maintainability, and ease of contribution.

New Calculator Module and Tests

  • Added a modular calculator implementation with support for basic arithmetic operations, infix evaluation, and error handling in calculator/pkg/calculator.py, along with a command-line interface in calculator/main.py and JSON output formatting in calculator/pkg/render.py. Comprehensive unit tests are provided in calculator/tests.py. [1] [2] [3] [4]

File Information Utility and Tests

  • Implemented a secure directory and file info retrieval utility in functions/get_files_info.py, with recursive traversal and error handling for invalid paths. Added corresponding unit tests in functions/tests.py and a standalone test script in test_get_files_info.py. [1] [2] [3]

Development Workflow Enhancements

  • Introduced a Makefile with recipes for environment setup, activation, running, linting, formatting, and testing. Added pre-commit configuration using Ruff and standard hooks in .pre-commit-config.yaml, and updated dependencies in pyproject.toml to include pre-commit and ruff. [1] [2] [3]

Documentation and Environment Configuration

  • Updated README.md to document Makefile usage, contribution guidelines, and project acknowledgments. Improved .env-example to clarify API key usage. [1] [2]

Gemini Client Improvements

  • Refactored main.py to improve Gemini API integration, including better argument parsing (with a --verbose flag), improved error handling, and more robust response formatting.

…var-name, updated gitignore file to better reflect the name cases of files to avoid issue in different operating systems, added some make recipes, added some dependencies like ruff and pre-commit, added config file for pre-commits hooks and synced uv.lock --@iamserda
…de-agent will work on and added unit tests for various operations for the calculator app --@iamserda
…ON output formatting; add file retrieval utility and corresponding tes, updated make, added basic and proper unit testing for path to folder llm will be allowed to work on. updated readme, makefile, gitignore --@iamserda
… limit; include tests for various file scenarios and add lorem.txt for testing --@iamserda
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant